Skip to content
This repository was archived by the owner on Jul 9, 2025. It is now read-only.

Conversation

@lei9444
Copy link
Contributor

@lei9444 lei9444 commented Mar 23, 2021

Description

issue
in 1.3.1 the eject runtime path is the relative path for the bot folder.
But in 1.4.0 the path is the relative path for the bot/settings.
So the bot created by 1.3.1 with custom runtime can't work.

fix
make the 1.3.1 and 1.4.0 use the same runtime path when do eject.
if the runtime is new runtime, still use the relative path for the bot/settings.

Task Item

#minor
closes #6519

Screenshots

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.008%) to 52.712% when pulling 245616e on lei9444:runtimepath into 294514f on microsoft:main.

@boydc2014 boydc2014 added the Approved to merge approved, waiting to be merged label Mar 23, 2021
@KarlXOL
Copy link

KarlXOL commented Mar 23, 2021

Thanks, this was helpful.
I've recreated the runtime in the meantime. This made bot creation and publishing work again.

Thanks
Karl

@boydc2014 boydc2014 merged commit 4dd99f4 into microsoft:main Mar 24, 2021
cwhitten pushed a commit that referenced this pull request Mar 31, 2021
Co-authored-by: Dong Lei <donglei@microsoft.com>
@cwhitten cwhitten mentioned this pull request May 20, 2021
benbrown pushed a commit to benbrown/BotFramework-Composer that referenced this pull request May 24, 2021
Co-authored-by: Dong Lei <donglei@microsoft.com>
benbrown pushed a commit that referenced this pull request Jun 11, 2021
Co-authored-by: Dong Lei <donglei@microsoft.com>
lei9444 added a commit to lei9444/BotFramework-Composer-1 that referenced this pull request Jun 15, 2021
Co-authored-by: Dong Lei <donglei@microsoft.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Approved to merge approved, waiting to be merged

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Start bot failed - Upgrade to composer 4.0

4 participants